Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

style: let the reader decide if something is simple or easy (upstream branch) #1201

Merged
merged 8 commits into from
Sep 10, 2024

Conversation

honzajavorek
Copy link
Collaborator

@honzajavorek honzajavorek commented Sep 10, 2024

Closes #1071. It's the same PR, but from an upstream branch, not from my fork. As discussed on Slack, this works around unauthorized npm token on CI.

I also rebased the original branch on top of current master, hopefully without mistakes. Otherwise the changes should be the same.

@honzajavorek
Copy link
Collaborator Author

So I went through the endless diff myself now, once again, just to be super sure I didn't mess up during the rebase. All seems good 🙏 @TC-MO can I have one more rubber stump here, please? (I know you've already approved the previous one, but GitHub won't allow me to merge this without explicit review.)

@honzajavorek honzajavorek added the t-academy Issues related to Web Scraping and Apify academies. label Sep 10, 2024
@honzajavorek honzajavorek merged commit 069d356 into master Sep 10, 2024
8 checks passed
@honzajavorek honzajavorek deleted the honzajavorek/simply2 branch September 10, 2024 12:40
@honzajavorek
Copy link
Collaborator Author

Thank you so much!

cat-highfive

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t-academy Issues related to Web Scraping and Apify academies.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants